Skip to content

src: remove license headers for new node_profiling files#63066

Open
legendecas wants to merge 1 commit intonodejs:mainfrom
legendecas:profiling_license
Open

src: remove license headers for new node_profiling files#63066
legendecas wants to merge 1 commit intonodejs:mainfrom
legendecas:profiling_license

Conversation

@legendecas
Copy link
Copy Markdown
Member

Remove unnecessary license headers for the new node_profiling files.

Refs: #63008 (comment)
Refs: #62273

@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. labels May 1, 2026
Signed-off-by: Chengzhong Wu <legendecas@gmail.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented May 1, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 89.65%. Comparing base (9ab960a) to head (8ec051d).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #63066      +/-   ##
==========================================
- Coverage   89.65%   89.65%   -0.01%     
==========================================
  Files         712      712              
  Lines      220498   220498              
  Branches    42285    42276       -9     
==========================================
- Hits       197693   197689       -4     
+ Misses      14673    14663      -10     
- Partials     8132     8146      +14     
Files with missing lines Coverage Δ
src/node_profiling.cc 81.33% <ø> (ø)
src/node_profiling.h 100.00% <ø> (ø)

... and 46 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@legendecas legendecas added the request-ci Add this label to start a Jenkins CI on a PR. label May 2, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label May 2, 2026
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants